Skip to content

Fix Qt 6 include conflict - #527

Open
Paulchen-Panther wants to merge 2 commits into
jgarff:masterfrom
Paulchen-Panther:master
Open

Fix Qt 6 include conflict#527
Paulchen-Panther wants to merge 2 commits into
jgarff:masterfrom
Paulchen-Panther:master

Conversation

@Paulchen-Panther

Copy link
Copy Markdown
Contributor

When adding the library in Cmake (target_include_directories) as submodules, there are conflicts with the compiler detection and your version file under Qt 6. Under Qt 5 the problem does not exist.

Therefore here is a PR that renames your version file to ".version". There are no changes in the core of your library by this PR.

Thank you very much.
Paulchen

@Paulchen-Panther

Copy link
Copy Markdown
Contributor Author

@jgarff
Would it be ok if I renamed the file “version” to “ws281x_version.h”?

@Paulchen-Panther

Copy link
Copy Markdown
Contributor Author

@jgarff
Is the change okay?

@Lord-Grey

Copy link
Copy Markdown

Hi @jgarff
Any chance to get this PR merged that the whole community can benefit from the great library and the latest updates?

Many thanks in advance!!!

@iam12pm

iam12pm commented Dec 16, 2024

Copy link
Copy Markdown

Hi @jgarff are there any news regarding the timeline for this mr to move to the master? Community is really waiting for this.

Lord-Grey added a commit to hyperion-project/rpi_ws281x that referenced this pull request Jan 7, 2025
@konstantyp

Copy link
Copy Markdown

@jgarff any news please ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants